Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mysqli #340

Closed
wants to merge 1 commit into from
Closed

mysqli #340

wants to merge 1 commit into from

Conversation

jlillyreed
Copy link

Since mysql_connect is deprecated, mysqli should be used instead

Since mysql_connect is deprecated, mysqli should be used instead
@xurizaemon
Copy link
Member

This relates to CRM-17789

I am willing to review this PR.

@colemanw
Copy link
Member

@xurizaemon can you do so early this week so we can get it merged?

@xurizaemon
Copy link
Member

xurizaemon commented May 14, 2016

Seems reviewable. Relates also to CRM-17789 & @civicrm/civicrm-core#8213 etc.

@JoeMurray
Copy link

Heh Jamil, as Release Manager this month, I'm trying to recruit people to help pare down the backlog of almost 100 PRs, some going back to last summer. I'm wondering if you would be able to help QA another PR if I got someone to QA this PR?

@jlillyreed
Copy link
Author

Hey @JoeMurray I would certainly be up for that

@JoeMurray
Copy link

@mollux and @jlillyreed could you two collaborate on this PR and #384 ? Thx!

@seamuslee001
Copy link
Contributor

@eileenmcnaughton @totten can one of you close this as there was a similar but alternate route that went down to ensure backward compatibility

@colemanw colemanw closed this Oct 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants